Conversation
Signed-off-by: Mridul Tailor <[email protected]>
PR SummaryLow Risk Overview Updates CI ( Reviewed by Cursor Bugbot for commit 4234e42. Bugbot is set up for automated code reviews on this repo. Configure here. |
| - name: Run Tests | ||
| run: node --run test |
There was a problem hiding this comment.
We should upload to codecov, no?
There was a problem hiding this comment.
Sure, I can add that. Just want to check, does the repo already have Codecov enabled on the org side?
Co-authored-by: Matt Cowley <[email protected]> Signed-off-by: Mridul Tailor <[email protected]>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d48f408. Configure here.

Closes #789
Summary
Sets up code coverage reporting using
@vitest/coverage-istanbul.@vitest/coverage-v8was attempted first but is incompatible with the Cloudflare Workers Vitest pool, it depends onnode:inspectorwhich is unavailable in that runtime. Istanbul is the supported alternative per the Cloudflare docs.Changes
package.json: addstest:coveragescriptvitest.config.ts: enables Istanbul coverage with text/lcov reporters and modest line/function thresholds.github/workflows/test.yml: CI now runstest:coverage.gitignore: addscoverage/to prevent generated artifacts from being committedpackage-lock.json: updated for@vitest/[email protected]